Skip to content

fix typos in plugin_uncore log message and plugin_video docstring#859

Open
xiejing-dev wants to merge 1 commit into
redhat-performance:masterfrom
xiejing-dev:master-2
Open

fix typos in plugin_uncore log message and plugin_video docstring#859
xiejing-dev wants to merge 1 commit into
redhat-performance:masterfrom
xiejing-dev:master-2

Conversation

@xiejing-dev

@xiejing-dev xiejing-dev commented Jun 10, 2026

Copy link
Copy Markdown

fix typos

@xiejing-dev
fix typos in plugin_uncore log message and plugin_video docstring
be9bffb 
Signed-off-by: xiejing <xiejing@kylinos.cn>
@coderabbitai

coderabbitai Bot commented Jun 10, 2026 •
edited
Loading

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Enterprise

Run ID: 84982be6-cd6d-48bc-b4a0-eb93f33370df

📥 Commits

Reviewing files that changed from the base of the PR and between 0eb28ac and be9bffb.

📒 Files selected for processing (2)
  • tuned/plugins/plugin_uncore.py
  • tuned/plugins/plugin_video.py
📝 Walkthrough

Summary by CodeRabbit

  • Bug Fixes

    • Corrected informational logging message for minimum frequency validation to display accurate reference values.

  • Documentation

    • Fixed spelling error in Radeon DPM option documentation.

Walkthrough

Two trivial corrections across plugin modules: a log message variable reference is fixed in the uncore plugin, and a documentation spelling error is corrected in the video plugin.

Changes

Minor Corrections

Layer / File(s) Summary
Uncore plugin log message variable reference
tuned/plugins/plugin_uncore.py		 In _validate_khz_value, the informational log message that reports when min_freq_khz is below the initial minimum is corrected to reference initial_min_freq_khz instead of initial_max_freq_khz.
Video plugin documentation spelling
tuned/plugins/plugin_video.py		 The VideoPlugin module docstring is corrected to spell the Radeon DPM option name as dpm-performance instead of the misspelled dpm-perfomance.

🎯 1 (Trivial) | ⏱️ ~2 minutes

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (4 passed)
Check name Status Explanation
Title check ✅ Passed The title accurately describes the main changes: fixing typos in both plugin_uncore log message and plugin_video docstring.
Description check ✅ Passed The description 'fix typos' is minimal but directly related to the changeset, which contains typo corrections in two plugin files.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@xiejing-dev